Remove the multiple-tab/windows-supported APIs

We'll have the caller do these instead via the populate-popup signal.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Rebuild the popup menu every time it is used

We'll need to rebuild it every time for the populate-popup signal.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

New signal populate-popup

Instead of conditionally creating contextual menu items like 'Show file properties',
we'll instead let the caller populate the popup menu, as is done for GtkEntry.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove gtk_places_sidebar_set_show_cwd(), implement that in the file chooser

This removes more idiosyncratic API from the sidebar.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove the GError from the shortcut-related functions

See https://bugzilla.gnome.org/show_bug.cgi?id=650363#c8 and the next comment; this
argument just makes the API harder to use without real benefit.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Rename signal location-selected to open-location

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove another to-do

Implement the Search keyboard shortcut

We still lack a button to activate Search, like in Nautilus, but that will come later.

And with this, the last REMOVE_FOR_PLACES_SIDEBAR is gone!

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove unused code

Go back to our machinery for Recent files instead of reading recent:/// as-is

Reading recent:/// as-is is probably fine for OPEN mode.  But for SAVE mode, we want to present
a list of recent directories.  So, we go back to our old machinery for now; in the future
we'll probably just use recent:/// and extract the list of recent directories from there.

This also fixes showing the Recent item as highlighted when the file chooser starts up.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove an obsolete function

Implement the Alt-number quick bookmark shortcuts

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Implement the Desktop keyboard shortcut

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Implement the Home keyboard shortcut

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Implement the 'Add to bookmarks' command as before

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove an obsolete function

Implement the application-side shortcuts in GtkFileChooserDefault

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

New machinery to add application-specific shortcuts

The file chooser needs them for gtk_file_chooser_add_shortcut_folder() and friends.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

One less to-do

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove the old shortcuts pane code from GtkFileChooserDefault

Oh, the giant void.
I stare into it
and it makes me afraid
that I've screwed up.

But I know I haven't.
Because any remaining bugs
will be yours to fix.

Not mine.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Allow not setting a changed callback on GtkBookmarksManager

Remove obsolete comment

Sync from Nautilus commit bde2536ee72672c4cd58af7cbb3cf42a42962c21

Use ellipsis character instead of dots

Sync from nautilus commit b2e8339d9f49770db0f6a523c8bf77b11d7eace3

Simplify setting eject icon

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Sync from nautilus commit 0177dd24a5f7f9f5302745a865eb970a35faa78c

Add list of pending things in GtkPlacesSidebar

Remove comments on things that Nautilus no longer needs to do

Use the right signal id

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Removed unneeded signal initiated-unmount

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Oops, forgot to commit this header file.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Oops, signals returning an int must be G_SIGNAL_RUN_LAST

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Port code to add a bookmark for a volume

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

New API to set whether URI drops are accepted

The file chooser will not accept dropping URIs in a shortcut item.  However, Nautilus will,
to perform normal DnD for files.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Emit drag-perform-drop when we need the caller to do the drop action

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

New signal drag-perform-drop

The caller is supposed to take action with the specified URIs.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

New signal drag-action-ask

We'll ask the caller to present a popup menu (or whatever) with the possible drag actions.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Emit drag-action-requested and feed the result to gdk_drag_status()

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

New signal drag-action-requested

We'll use this signal to ask the caller about what drag action to indicate
when something is being hovered on an item in the places sidebar.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Free URI lists

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Rename function for clarity

build_selection_list() -> build_uri_list()

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove the menu items to select recent-files or CWD

Per the gtk-devel meeting on 2012/11/29, we'll not have a UI in the file chooser dialog
to select between those modes.  Instead, we'll show that key in gtweaktool.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Ahem, read radio menu items like I knew how to do it

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Restore the originally-selected location when dynamically updating the bookmarks

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Oops, actually update the internal location of the places sidebar prior to notifying the caller

This is so that the sidebar will already know the URI of the location it is showing.  When
the caller asks the sidebar to change back to the same location, the call will be idempotent.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Highlight the currently-viewed location in the places sidebar

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Turn gtk_places_sidebar_set_current_uri() into _set_current_location()

To be consistent with the ::location-selected signal, which emits a GFile, we now also use a GFile
instead of a string URI in the inbound API.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Actually switch to recent:/// in its key shortcut handler

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Allow switching to recent:/// even though GIO thinks it is not a native file

That URI is not a native file, and so it would not show up if local_only=TRUE.
Add a special case for it so that recent:/// can always be visited.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Update the widgets until all of them have been created

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Add menu items to the places sidebar to configure the startup mode

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Switch to Recent Files or $CWD at startup

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Add a new org.gtk.SettingsFileChooser startup-mode settings key

It has two possible values, 'recent' and 'cwd'.  We will use these to determine
whether to set the default starting mode, if a folder has not been pre-set,
to showing the Recent Files list or the current working directory.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

New way to show  in the sidebar

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Note about missing empty-trash functionality when unmounting a volume.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Fix code to unmount volumes

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Copy code from Nautilus to mount volumes

Instead of having a nested-callback scheme (to preserve internal API compatibility),
we use a direct callback from g_volume_mount().

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Set GTK_STYLE_CLASS_SIDEBAR on the sidebar

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove some more comments on things that Nautilus needs to do

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove some notes on things that Nautilus should do but already does

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Add a show-file-properties signal for Nautilus to use

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Note on more things that Nautilus needs to do

Present error dialogs from the places sidebar

Oops, reorder the bookmarks list correctly

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Enable code to reorder bookmarks by dragging

Sync from Nautilus commit b72d57b038e50172fac2fafdc6c26660378aab34

No changes (except a whitespace fix), for we already had a copy of
special_directory_get_gicon().

Add comment about missing way to get symbolic icons for bookmarks
Nautilus got the ability to do that in commit 0ed400b9c1692e42498bff3c10780073ec137f63.
Maybe we should just copy that code to here later.

Sync from Nautilus: bookmark: remove unused GIcon argument to nautilus_bookmark_new()
We always pass NULL these days.
This is from nautilus commit 7aad4a33814e6e7fae5e762882db43452a575a83

Sync from Nautilus - Fix leaks
https://bugzilla.gnome.org/show_bug.cgi?id=683769
This is from nautilus commit a755aa27c636561a1ab41066484cf9bcc97667f9

Merge master into places-sidebar to keep up with development

Update Czech translation

wayland: Update to reflect protocol changes

Update Galician translations

Updated Slovenian translation

Fix duplicate columns in filechooser entry completion

This partically reverts commit
331bba1ad6c5d9535fc5a827b91019f5824e25a7, which broke documented
behaviour.

menubutton: don't refer to the non-existant menu property

7c6454246e7fa181d9e16f3520bc6d3765168f40 removed the property, but
forgot to change the name in a g_object_notify().

Updated Serbian translation

Updated Indonesian translation

Updated Spanish translation

level-bar: Fix typo

scrolledwindow: set GDK_EXPOSURE_MASK on the overshoot window

Currently we use gtk_style_context_set_background() when the state flags
change in order to propagate the background color to the overshoot
window, but this is actually only needed because the window doesn't get
expose events, since we always draw a full background in draw().
This also fixes some problems when the GdkWindow of the scrolled
window's child is composited, as seen in oxygen-gtk3.

https://bugzilla.gnome.org/show_bug.cgi?id=686265

level-bar: add support for RTL locales

https://bugzilla.gnome.org/show_bug.cgi?id=684288

level-bar: add an "inverted" property like GtkProgressBar

https://bugzilla.gnome.org/show_bug.cgi?id=684288

Add gdk version macros for 3.8

https://bugzilla.gnome.org/show_bug.cgi?id=684288

GtkMenuButton: remove menu property as it is replaced by popup.

See that it was already announced to be removed before 3.6.0
but we forgot.

cssshadowsvalue: handle gtk_css_value_transition returning NULL

The implementation of transition for GtkCssShadowValue can return NULL
at least when the two values have a different inset; all other parts of
the GTK/CSS machinery (e.g. GtkCssArrayValue) handle this by returning
NULL too. Instead, GtkCssShadowsValue was returning an invalid value,
where "len" was set, but some values in the array were NULL, which would
lead to a segfault when this value is later evaluated by the compute
function.

Fix this by making GtkCssShadowsValue return NULL if a shadow transition
fails, like GtkCssArrayValue does.

https://bugzilla.gnome.org/show_bug.cgi?id=686013

reftests: fix the linear-gradient reftest

Don't use a repeating linear gradient, since it can't be easily
reftested against a non-repeating one for the reasons described in the
test header.
Instead, add a separate test for repeating gradients (against another
repeating gradient).

This makes the test pass, so it can be added to the Makefile now.

cssshadow: plug a cairo_surface_t leak

We were never destroying the cairo surface we use for blurring, which
would lead to a huge leak.

https://bugzilla.gnome.org/show_bug.cgi?id=686209

csskeyframes: unref GtkCssValues when free-ing

We assume a reference to all the GtkCssValues we store, so we need to
release it when free-ing.

gradient: plug a GtkCssValue refleak

Since _gtk_symbolic_color_resolve_full() returns a reference to a
GtkCssValue.

csscomputedvalues: plug a refleak

_gtk_css_keyframes_compute() returns a reference to a GtkCssKeyframes,
and _gtk_css_animation_new() takes another reference.

bump version

3.6.1

Fix css parser tests

Parsing a shorthand background property was running into unexpected
errors when trying position values where there were none. To fix this,
introduce a try_parse variant of the position parse function that
silently returns NULL.

Fix 'make check'

appchooserwidget: check for g_app_info_should_show()

This ensures NoDisplay=true or NotShowIn/OnlyShowIn values in the
desktop file are respected.

a11y: fix private strict regression

Commit 0bbfcc249191026a1ee822545398ec6e5e36f7f4 added a private struct
for GtkImageAccessible, but forgot to call g_type_class_add_private().

a11y: fix the build

Fix a typo introduced in 0ad379708f79dbff916b1fcd431d3e3114bf8e25

Add private structs to all remaining headers

These are not used currently. Just put them in now in
case we need them in the future.

GtkWidgetAccessible: add a private struct

Move instance fields to a private struct, in preparation
for installing a11y headers.
This also required removing access to GtkWidgetAccessible innards
from several accessible implementations.

GtkTreeViewAccessible: add a private struct

Move instance fields to a private struct, in preparation
for installing a11y headers.